Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge 9 -> main #1077

Merged
merged 20 commits into from
Oct 9, 2024
Merged

Merge 9 -> main #1077

merged 20 commits into from
Oct 9, 2024

Conversation

iche033
Copy link
Contributor

@iche033 iche033 commented Oct 4, 2024

➡️ Forward port

Port gz-rendering9 to main

Branch comparison: main...gz-rendering9

Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

iche033 and others added 19 commits August 25, 2024 18:06
Signed-off-by: Van Malleghem Antoine <[email protected]>
Signed-off-by: Van Malleghem Antoine <[email protected]>
Signed-off-by: Antoine Van Malleghem <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Van Malleghem Antoine <[email protected]>
Revert "Fix using lightmap and emissive map together (#1047)"
By adding the missing DL dependency

Signed-off-by: Silvio Traversaro <[email protected]>
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Windows CI is broken but we can fix this in a follow up PR

usage: colcon [-h] [--log-base LOG_BASE] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,graph,info,list,metadata,test,test-result,version-check}
              ...
colcon: error: unrecognized arguments: -A x64 -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF

@iche033
Copy link
Contributor Author

iche033 commented Oct 4, 2024

Windows CI is broken but we can fix this in a follow up PR

usage: colcon [-h] [--log-base LOG_BASE] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,graph,info,list,metadata,test,test-result,version-check}
              ...
colcon: error: unrecognized arguments: -A x64 -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=OFF

Not sure why it's working on gz-rendering9 branch and not main. Took a quick look, colcon and pixi versions are the same. I also tried merging with #1052 but didn't help. Will continue looking into it

Signed-off-by: Ian Chen <[email protected]>
@iche033
Copy link
Contributor Author

iche033 commented Oct 9, 2024

merging this first. Will address windows CI github actions separately

@iche033 iche033 merged commit 115a179 into main Oct 9, 2024
8 of 10 checks passed
@iche033 iche033 deleted the merge_9_main_20241004 branch October 9, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

8 participants